Skip to content

Conversation

@Toflar
Copy link
Member

@Toflar Toflar commented Aug 14, 2025

No description provided.

@Toflar Toflar requested a review from fritzmg as a code owner August 14, 2025 14:29
@Toflar
Copy link
Member Author

Toflar commented Sep 15, 2025

Ping @fritzmg

Copy link
Contributor

@fritzmg fritzmg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm - however, what I am kind of missing is the intended or typical usage. You will usually have something like this, right?

// In some controller
$job = $this->jobs->createJob('data_export');
$message = new DataExportMessage($someData, $job->getUuid());
$this->messageBus->dispatch($message);
// In the DataExportMessageHandler
$job = $this->jobs->getByUuid($message->getJobUuid());
$this->jobs->persist($job->markPending());

// Do some work …

$this->jobs->persist($job->markCompleted());

@Toflar
Copy link
Member Author

Toflar commented Sep 15, 2025

Yes, if you want to. Could also be a cronjob though etc., yes.

@Toflar
Copy link
Member Author

Toflar commented Nov 3, 2025

Can we get this merged? It's still completely undocumented at the moment although this PR has been here for almost a quarter now...

@fritzmg
Copy link
Contributor

fritzmg commented Nov 3, 2025

I'd still prefer it if we had examples. But if neither of us has the time, I will merge it as is.

@fritzmg fritzmg merged commit 5c43e04 into main Nov 3, 2025
@fritzmg fritzmg deleted the feature/jobs-framework branch November 3, 2025 14:17
@fritzmg
Copy link
Contributor

fritzmg commented Nov 3, 2025

Thank you @Toflar

@Toflar
Copy link
Member Author

Toflar commented Nov 3, 2025

It's super experimental still anyway. I think it'll make much more sense to explain it with regard to "Look at how the core does it in xy" once we use it properly ourselves which might happen in v5.7 😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants